Skip to content

Sync sdk 25.06.00 to hal nxp #576

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

ZhaoxiangJin
Copy link
Collaborator

Sync sdk 25.06.00 to hal nxp

@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , please take a look at this PR #574 that is requiring update to newer version of the mipi-dsi driver.

@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch from 578fb8c to 5c335ba Compare July 24, 2025 06:58
@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , another PR that is updating SDK drivers. Is this covered as part of this update?
#581

@ZhaoxiangJin
Copy link
Collaborator Author

@ZhaoxiangJin , another PR that is updating SDK drivers. Is this covered as part of this update? #581

Not covered, PR can merge first, then i will drop irqsteer_1 driver in my PR.

@zejiang0jason
Copy link
Contributor

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

@zejiang0jason
Copy link
Contributor

Are the RT1180 board level jlink script needed?

@zejiang0jason
Copy link
Contributor

should the binary files in mcux-sdk-ng/components/conn_fwloader/fw_bin/ be placed in zephyr/modules/hal/nxp/zephyr/blobs?

@ZhaoxiangJin
Copy link
Collaborator Author

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

Why move sdk files to zephyr repo,

Are the RT1180 board level jlink script needed?

yes, see https://github.com/zephyrproject-rtos/zephyr/blob/main/boards/nxp/mimxrt1180_evk/board.cmake

@ZhaoxiangJin
Copy link
Collaborator Author

should the binary files in mcux-sdk-ng/components/conn_fwloader/fw_bin/ be placed in zephyr/modules/hal/nxp/zephyr/blobs?

The conn_fwloader CMakeList.txt needs these binary files, move these binary files requires modify conn_fwloader CMakeList.txt

@ZhaoxiangJin
Copy link
Collaborator Author

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

Why move SDK files to zephyr repo? Only glue code should be placed in zephyr repo.

@zejiang0jason
Copy link
Contributor

Can the mcux-sdk-ng/board folder be moved to Zephyr repo?

Why move SDK files to zephyr repo? Only glue code should be placed in zephyr repo.

Move to the Zephyr repo board folder, they are board specific files.

@zejiang0jason
Copy link
Contributor

Hi @ZhaoxiangJin , please note don't miss this code during migration: #577. thanks.

@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch from b77ba07 to 905aff8 Compare July 30, 2025 06:23
@ZhaoxiangJin ZhaoxiangJin marked this pull request as ready for review July 30, 2025 06:24
This cmake code is not needed for zephyr, and is specific for
MCUX SDK. SDK also applied this patch.

Signed-off-by: Zhaoxiang Jin <[email protected]>
1. Remove MCXA166 and MCXA276 as they are renamed to
MCXA346 and MCXA266.
2. Add MCXA266 and MCXA366 header and pinctrl
3. Rename MCXA166 to MCXA346.

Signed-off-by: Peter Wang <[email protected]>
Signed-off-by: Zhaoxiang Jin <[email protected]>
The latest version of the caam driver causes run
issues on the RT1170/RT1160 platform, using an older
version for now, and have contacted the SDK driver
owner to resolve the issue.

Signed-off-by: Zhaoxiang Jin <[email protected]>
@ZhaoxiangJin ZhaoxiangJin force-pushed the feature/sync-sdk-25.06.00-to-hal-nxp branch from 6c0d41d to 53fea4f Compare August 11, 2025 05:33
@ZhaoxiangJin
Copy link
Collaborator Author

Update:
removed boards folder, moved board specific files to zephyr repo.

@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , please include #571 as well in your update

@mmahadevan108
Copy link
Collaborator

@ZhaoxiangJin , please include #575 in your update

@mmahadevan108
Copy link
Collaborator

Another PR that is changing mcux-sdk-ng #578

@mmahadevan108
Copy link
Collaborator

Another PR changing mcux-sdk-ng: #573

Zhiqiang-Hou and others added 5 commits August 14, 2025 09:09
The comment sign character is '//' on ARMv8 AARCH64, however it's
'@' on other ARM architectures.
So the current definitions of AT_NONCACHEABLE_SECTION_ALIGN
and AT_NONCACHEABLE_SECTION will result in the following error on
AARCH64.

  /tmp/cc96AQ7j.s: Assembler messages:
  /tmp/cc96AQ7j.s:1020: Error: junk at end of line, first unrecognized character is `@'

Signed-off-by: Hou Zhiqiang <[email protected]>
Signed-off-by: Stephane Viau <[email protected]>
Signed-off-by: Jiafei Pan <[email protected]>
NETC driver is not compatible with struct definition, for example will
report the following build error like:
modules/hal/nxp/mcux/mcux-sdk-ng/drivers/netc/fsl_netc_timer.h:369:20:
error: 'ENETC_PF_TMR_Type' has no member named 'TMR_TEVENT'

Signed-off-by: Jiafei Pan <[email protected]>
This Errata is not available on i.MX 95.

Signed-off-by: Jiafei Pan <[email protected]>
Updated netc driver to the latest SDK commit.

5309b12e [MCUX-80906] drivers: netc: enable ring0 for EP supporting tag

Signed-off-by: Yangbo Lu <[email protected]>
@ZhaoxiangJin
Copy link
Collaborator Author

Hello @mmahadevan108, #581, #577, #575, #571 have picked to my PR. #573, #574, #578 are pending in review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants